Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add readers and writers instances for cats-collections #286

Closed

Conversation

Roman-Statsura
Copy link
Contributor

Add readers and writers instances for cats-collections

@Roman-Statsura
Copy link
Contributor Author

I have problems with some collections, such as Range: how does look like Range(1,5) in Json? I have two option: [1,2,3,4,5] and [1, 5]. Which option is correct? How to read Range from Json? It might not be worth implementing instances for this and anyone can do it themselves?

build.sbt Outdated
@@ -9,7 +9,7 @@ lazy val scala3 = "3.3.0"
ThisBuild / scalaVersion := scala3

lazy val commonSettings = Seq(
version := "0.28.4",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you rebase commit on a master branch?
There is no more need to update version by hand

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok I did it

@dos65
Copy link
Collaborator

dos65 commented May 15, 2024

@Roman-Statsura Sorry for delay!
I just realized that it should be better to inroduce an additional model tethys-cats-collection rather that extendening existing one with typelevel/cats-collections. Could you do that?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants